Skip to content

Stronger faster better - #17

Merged
dignifiedquire merged 1 commit into
masterfrom
stronger-faster-better
Jun 6, 2016
Merged

dignifiedquire merged 1 commit into
masterfrom
stronger-faster-better

Conversation

@dignifiedquire

Copy link
Copy Markdown
Member

Found issues when distributing blocks in networks with >= 3 nodes. This adds tests for that and hopefully a fix soon :)

cc @diasdavid @haadcode

@dignifiedquire

Copy link
Copy Markdown
Member Author

Mostly works, but socket hung up errors are making life hard :/

@daviddias

Copy link
Copy Markdown
Member

Pasting the notes from the etherpad. Later we should do also the same tests on full js-ipfs nodes, so that we account for every possibility

bitswap tests

  • script to spawn a node, given a config file, creates a random file every X seconds and prints the hash (so that it can be requested
  • spawn 1 node on the main thread with TCP
  • dial to the other down
  • fetch all of the blocks the other node outputs
  • spawn 5 nodes in separate process and repeat the above
  • spawn 10 nodes and ..
  • spawn 20 nodes
  • repeat the previous proc
  • note: measure time (avg, max and min) of fetching the block and memory usage in the beginning and end

timing data in different events

  • use logs (debug)

@dignifiedquire

Copy link
Copy Markdown
Member Author

@diasdavid as not all of this will happen in this PR I moved the notes to #8

@daviddias

Copy link
Copy Markdown
Member

@dignifiedquire dignifiedquire changed the title [WIP] Stronger faster better Stronger faster better Jun 6, 2016
- send messages to all connected peers on start up
- ensure b58 endcoded strings are used everyhwere
- update dependencies
- add improved network tests
@dignifiedquire
dignifiedquire force-pushed the stronger-faster-better branch from a2e9aa7 to 6563398 Compare June 6, 2016 10:41
@dignifiedquire
dignifiedquire merged commit fc128b0 into master Jun 6, 2016
@dignifiedquire
dignifiedquire deleted the stronger-faster-better branch June 6, 2016 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants